Skip to content

[stable32] fix(theming): display error messages on image upload and delete previous image only after validation#58322

Open
backportbot[bot] wants to merge 2 commits intostable32from
backport/58003/stable32
Open

[stable32] fix(theming): display error messages on image upload and delete previous image only after validation#58322
backportbot[bot] wants to merge 2 commits intostable32from
backport/58003/stable32

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Feb 13, 2026

Backport of #58003

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

…ous image only after validation

Signed-off-by: Cristian Scheid <cristianscheid@gmail.com>
Signed-off-by: Cristian Scheid <cristianscheid@gmail.com>
@cristianscheid cristianscheid marked this pull request as ready for review February 13, 2026 15:27
@cristianscheid cristianscheid requested a review from a team as a code owner February 13, 2026 15:27
@cristianscheid cristianscheid requested review from Altahrim, ArtificialOwl and icewind1991 and removed request for a team February 13, 2026 15:27
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

Copy link
Member

@provokateurin provokateurin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about the frontend changes?

@cristianscheid
Copy link
Contributor

What about the frontend changes?

@provokateurin Sorry, forgot to mention this here, but frontend changes are not required for stable32. The current implementation already handles the server response properly and displays the appropriate message to users. I've retested to confirm it works as expected:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants